Update @github/copilot to 1.0.51-3#1351
Merged
Merged
Conversation
- Updated nodejs and test harness dependencies - Re-ran code generators - Formatted generated code
stephentoub
approved these changes
May 20, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the monorepo’s @github/copilot dependency to 1.0.51-3 and re-runs the various language code generators so the SDKs stay in sync with the updated Copilot CLI/schema wire protocol.
Changes:
- Bumped
@github/copilotto^1.0.51-3in Node packages and updated lockfiles accordingly. - Regenerated RPC/session-event types across Node.js, Python, Go, Rust, and .NET.
- Picked up schema-driven model updates (notably: new slash-command “select-subcommand” result types, additional enum docs, and
ttftMs→timeToFirstTokenMsrenames).
Show a summary per file
| File | Description |
|---|---|
| test/harness/package.json | Bumps @github/copilot devDependency version. |
| test/harness/package-lock.json | Lockfile update for @github/copilot@1.0.51-3 and platform packages. |
| nodejs/package.json | Bumps @github/copilot dependency version. |
| nodejs/package-lock.json | Lockfile update for @github/copilot@1.0.51-3 and platform packages. |
| nodejs/samples/package-lock.json | Samples lockfile updated to align @github/copilot version. |
| nodejs/src/generated/session-events.ts | Regenerated session-event types; includes ttftMs rename and expanded doc comments. |
| nodejs/src/generated/rpc.ts | Regenerated RPC types; includes new slash-command result variant and expanded doc comments. |
| python/copilot/generated/session_events.py | Regenerated session-event models; includes ttftMs rename and expanded enum comments. |
| python/copilot/generated/rpc.py | Regenerated RPC models; adds “select-subcommand” slash-command result support. |
| rust/src/generated/session_events.rs | Regenerated session-event Rust types; includes ttftMs rename and enum doc updates. |
| rust/src/generated/api_types.rs | Regenerated Rust RPC/api types; adds slash-command “select-subcommand” result types/variant. |
| go/rpc/zsession_events.go | Regenerated Go session-event types; includes ttftMs rename and enum doc comments. |
| go/rpc/zrpc.go | Regenerated Go RPC types; adds slash-command “select-subcommand” result types and enum docs. |
| go/rpc/zrpc_encoding.go | Updates Go RPC JSON (un)marshalling for the new slash-command result variant. |
| dotnet/src/Generated/SessionEvents.cs | Regenerated .NET session-event models; includes ttftMs rename and URI annotations. |
| dotnet/src/Generated/Rpc.cs | Regenerated .NET RPC models; adds “select-subcommand” derived result and supporting types. |
Copilot's findings
Files not reviewed (6)
- go/rpc/zrpc.go: Language not supported
- go/rpc/zrpc_encoding.go: Language not supported
- go/rpc/zsession_events.go: Language not supported
- nodejs/package-lock.json: Language not supported
- nodejs/samples/package-lock.json: Language not supported
- test/harness/package-lock.json: Language not supported
- Files reviewed: 2/16 changed files
- Comments generated: 2
- nodejs/src/client.ts: Add 10s connection timeout to connectViaTcp() to prevent indefinite hangs on Windows when reconnecting after forceStop() - dotnet/test: Change MCP server command from 'echo' to 'true' in the Without_Args test. On Ubuntu, 'echo' (no args) outputs a bare newline that confuses the CLI's MCP JSON-RPC reader, causing a 2-minute hang. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
Cross-SDK Consistency ReviewThis PR regenerates code from the updated The one non-generated change is the 10-second TCP connection timeout added to
Rust and .NET would benefit from equivalent SDK-level timeouts to ensure a non-responsive CLI server doesn't cause an indefinite hang in those SDKs. This could be addressed in a follow-up PR.
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
Author
There was a problem hiding this comment.
Generated by SDK Consistency Review Agent for issue #1351 · ● 960.1K
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated update of
@github/copilotto version1.0.51-3.Changes
@github/copilotinnodejs/package.jsonandtest/harness/package.jsonscripts/codegen)Next steps
When ready, click Ready for review to trigger CI checks.